Fold the 2-BM tomoscan survey findings into Operations and Open questions - #647
Merged
Conversation
…ions DATA-9 and DATA-10 are settled by direct read against the live 2bmb deployment on 2026-08-11 (git diff of the deployed tomoscan checkout, content of the deployed areaDetector layout XML, a live caget of the timestamp PV), so their rows are deleted per this page's own confirmed-item convention and the evidence recorded in Operations. DATA-8 stays open: a real dropped-frame instance was found (a smoke test with theta absent), confirming the reader's designed handling, but the frequency question for production scans is unanswered. HOST-1/HOST-2 gain the concrete values found (arcturus's actual EPICS_CA_ADDR_LIST; the real file location, /local2 not /local1) and a note that whether CORA's host inherits them is asked of Francesco. New HOST-6 asks which host CORA should actually run on, prompted by finding a pre-staged, empty `cora` conda env on tomdet. Operations' storage-pipeline paragraph is corrected from /local1 to /local2, per a direct read of the live DetectorTopDir PV. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Coverage report
This PR does not seem to contain any modification to coverable code. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
git diffof the deployed tomoscan checkout, content of the deployed areaDetector layout XML, and a livecagetof the timestamp PV. Rows deleted per this page's own confirmed-item convention; evidence recorded in Operations.thetaabsent), confirmingDataExchangeScanReader's designed handling, but the frequency question for production scans is unanswered.EPICS_CA_ADDR_LIST; the real file location is/local2, not/local1), plus a note that whether CORA's host inherits them is being asked of the beamline contact.coraconda env on tomdet./local1to/local2, per a direct read of the liveDetectorTopDirPV.Test plan
mkdocs build --strict(CI's exact dep pins) — same 7 pre-existing warnings as main, all in files this change doesn't touch (contributing.md/security.md/run.md/talks.md); no new warningsNote: an earlier draft of this change marked DATA-9/DATA-10
Already done? = yesinstead of deleting the rows — caught against this page's own documented convention ("confirmed items get the row deleted") before commit and corrected.🤖 Generated with Claude Code